Skip to content

Fix test file versioning with content change#127

Merged
agritheory merged 3 commits intoversion-15from
fix_test_file_versioning_with_content_change
Apr 9, 2026
Merged

Fix test file versioning with content change#127
agritheory merged 3 commits intoversion-15from
fix_test_file_versioning_with_content_change

Conversation

@lauty95
Copy link
Copy Markdown
Collaborator

@lauty95 lauty95 commented Apr 9, 2026 •
edited
Loading

Fixes the failingtest_file_versioning_with_content_change test revealed by #126

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 9, 2026

📝 Draft Changelog Entry

Changelog

This update fixes file versioning functionality in cloud storage operations. An issue was resolved where file version tracking was not working correctly when file content changed. The S3 versioning feature has been enabled in the test environment to ensure proper version management, and related test cases have been updated to accurately validate this behavior.

This changelog entry was automatically generated by the Changelog Generator Action.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 9, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
__init__.py11554%13–14, 20–22
commands.py35350%4–6, 9–11, 17–20, 23, 26–27, 49–51, 53–54, 56, 65, 68–72, 75–76, 85–87, 89–90, 92, 94, 97
hooks.py140100% 
install.py36360%4–7, 10–11, 14–18, 21–27, 30–34, 36–39, 41–49
migration.py19611541%25, 46, 68–69, 88–89, 96, 115, 123–125, 158–159, 161–162, 166–168, 171–172, 187, 196–197, 199–203, 205–210, 212, 219–221, 223, 225–226, 228, 235–236, 238–242, 244–247, 249, 254–257, 259–268, 270–279, 281–282, 284–285, 287–288, 290–294, 298–301, 305, 307–308, 310, 312–314, 317–318, 320–321, 323–329, 331–332, 334, 336
cloud_storage
   __init__.py00100% 
cloud_storage/doctype
   __init__.py00100% 
cloud_storage/doctype/file_association
   __init__.py00100% 
   file_association.py30100% 
cloud_storage/doctype/file_version
   __init__.py00100% 
   file_version.py30100% 
cloud_storage/overrides
   file.py41413068%66, 76, 96, 119, 121, 136, 138, 145–147, 155–156, 171, 176, 183, 198, 256–265, 267–275, 286, 291–292, 300–302, 304, 309, 315, 317, 333, 337, 342, 350–351, 354, 357, 386, 431, 434, 473, 479, 485, 491, 497, 503, 513, 515–517, 521, 529–531, 533, 551, 564–568, 574–575, 577, 600–604, 616–617, 666, 669–678, 685–686, 688–690, 693–694, 699–701, 703, 708–711, 716, 718, 730–731, 733, 738–742, 744, 755–761
config
   __init__.py00100% 
   desktop.py330%4, 7–8
   docs.py660%4, 6–8, 12–13
tests
   conftest.py44197%54
   test_file.py173398%89, 174, 226
   test_file_preview.py630100% 
   test_local_storage.py610100% 
TOTAL106233468% 

@lauty95 lauty95 requested a review from agritheory April 9, 2026 18:03
@agritheory agritheory merged commit c35c4b7 into version-15 Apr 9, 2026
6 checks passed
@agritheory agritheory deleted the fix_test_file_versioning_with_content_change branch April 9, 2026 19:28
@agritheory agritheory mentioned this pull request Apr 9, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@agritheory agritheory agritheory approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lauty95 @agritheory